Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[data grid] Change test dom check from /jsdom/ to /jsdom|HappyDOM/ #15634

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

jedesroches
Copy link
Contributor

This change enables testing with Happy DOM without annoying warnings popping up. Fixes #10374.

Changelog

Detect HappyDOM like jsdom through the user agent string to avoid width and height warnings during tests.

This change enables testing with Happy DOM without annoying warnings
popping up. Fixes mui#10374.
@jedesroches jedesroches changed the title Change test dom check from /jsdom/ to /jsdom|HappyDOM/. [x-data-grid][hooks] Change test dom check from /jsdom/ to /jsdom|HappyDOM/. Nov 27, 2024
@mui-bot
Copy link

mui-bot commented Nov 27, 2024

Deploy preview: https://deploy-preview-15634--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against d57280e

@romgrk romgrk added component: data grid This is the name of the generic UI component, not the React module! needs cherry-pick The PR should be cherry-picked to master after merge v7.x labels Nov 28, 2024
@romgrk romgrk merged commit 8725903 into mui:master Nov 28, 2024
23 of 24 checks passed
Copy link

Cherry-pick PRs will be created targeting branches: v7.x

github-actions bot pushed a commit that referenced this pull request Nov 28, 2024
…15634)

Co-authored-by: Joachim Desroches <jo@thedesroches.net>
@oliviertassinari oliviertassinari changed the title [x-data-grid][hooks] Change test dom check from /jsdom/ to /jsdom|HappyDOM/. [x-data-grid][hooks] Change test dom check from /jsdom/ to /jsdom|HappyDOM/ Nov 28, 2024
@oliviertassinari oliviertassinari changed the title [x-data-grid][hooks] Change test dom check from /jsdom/ to /jsdom|HappyDOM/ [data-grid] Change test dom check from /jsdom/ to /jsdom|HappyDOM/ Nov 28, 2024
@oliviertassinari oliviertassinari changed the title [data-grid] Change test dom check from /jsdom/ to /jsdom|HappyDOM/ [data grid] Change test dom check from /jsdom/ to /jsdom|HappyDOM/ Nov 28, 2024
LukasTy pushed a commit to LukasTy/mui-x that referenced this pull request Dec 19, 2024
…ui#15634)

Co-authored-by: Joachim Desroches <jo@thedesroches.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module! needs cherry-pick The PR should be cherry-picked to master after merge test v7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Detect "HappyDOM" in userAgent string in same way as "jsdom"
4 participants